-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bsr bump spectree #7617
Bsr bump spectree #7617
Conversation
a8ede8a
to
8f2af86
Compare
29a27e9
to
fcfaa9c
Compare
add347d
to
73d7b84
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Top, merci pour ce chouette boulot. ❤️
fc0d003
to
2c1f26b
Compare
I checked, the operationId is not used by the frontend, this is not a breaking change for us
What's new: - spectree now handles form in requests - spectree considers empty body as not json. Therefore I added a header to tell spectree validators to expect json if body is None
Our implementation of generate_spec is basically the same as the one in the lib cf. https://github.com/0b01001001/spectree/blob/98f2e1d522c291165b2a6233416c26a224f2750b/spectree/response.py#L124
Can't believe it works!
I specifically commit that separately because I am not 100% confident this actually tests the same thing
2c1f26b
to
a666f12
Compare
Visit the preview URL for this PR (updated for commit 33308e8): https://pc-pro-testing--pr7617-bsr-bump-spectree-6lkuek4v.web.app (expires Wed, 16 Aug 2023 15:51:39 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
But de la pull request
Bump spectree
En effet, selon leur doc, pydantic v2 est supporté seulement à partir de la 1.2
Et on veut pydantic v2
Changelog de SpecTree: https://github.com/0b01001001/spectree/releases
Ticket Jira (ou description si BSR) : PC-23699
Vérifications